Skip to content

feat: support multiple versions of the pgrouting extension #1687

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 8 commits into from
Oct 6, 2025
Merged

feat: support multiple versions of the pgrouting extension #1687

merged 8 commits into from
Oct 6, 2025
+377 −61

Conversation

jfroche
Copy link
Collaborator

@jfroche jfroche commented Jul 1, 2025

Build multiple versions of the pgrouting extension on different PostgreSQL versions.
Add test for the extensions and their upgrade on PostgreSQL 15 and 17.
Run pg_regress for pgrouting before and after upgrade.

@jfroche jfroche requested review from a team as code owners July 1, 2025 16:14
hunleyd
hunleyd previously requested changes Aug 27, 2025
View reviewed changes
Copy link
Contributor

@hunleyd hunleyd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rebase to fix conflicts?

@jfroche jfroche force-pushed the multi-version-ext/pgrouting branch from 8dda7e0 to b4659bc Compare October 1, 2025 09:15
@hunleyd hunleyd self-requested a review October 1, 2025 16:47
@hunleyd hunleyd dismissed their stale review October 1, 2025 16:48

pls re-review

hunleyd
hunleyd approved these changes Oct 1, 2025
View reviewed changes
Copy link
Contributor

@hunleyd hunleyd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@samrose samrose force-pushed the multi-version-ext/pgrouting branch 2 times, most recently from f7f6917 to f6596bd Compare October 3, 2025 13:40
samrose
samrose requested changes Oct 6, 2025
View reviewed changes
Copy link
Collaborator

@samrose samrose left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

local infra testing

@samrose samrose force-pushed the multi-version-ext/pgrouting branch from e78106c to df73191 Compare October 6, 2025 19:46
samrose
samrose approved these changes Oct 6, 2025
View reviewed changes
Copy link
Collaborator

@samrose samrose left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tested and ready to merge

@samrose samrose merged commit d4be4ed into develop Oct 6, 2025
6 checks passed
@samrose samrose deleted the multi-version-ext/pgrouting branch October 6, 2025 23:05
hunleyd added a commit that referenced this pull request Oct 7, 2025
@hunleyd
Merge branch 'develop' of github.com:supabase/postgres into ansible_c…
130cb86 
…leanups

* 'develop' of github.com:supabase/postgres:
  feat: support multiple versions of the pgrouting extension (#1687)
  refactor(ansible): bring our ansible up to modern ansible-lint standards (#1819)
  feat: support multiple versions of the pg-graphql extension (#1761)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@samrose samrose samrose approved these changes

@hunleyd hunleyd hunleyd approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jfroche @samrose @hunleyd